Skip to content

metricbeat/filebeat: fix typos in iis, oracle, etcd, sql, decode_cef, and netflow modules#49643

Open
strawgate wants to merge 7 commits intoelastic:mainfrom
strawgate:strawgate/fix-typos-pr7-obs-infraobs-integration-experience
Open

metricbeat/filebeat: fix typos in iis, oracle, etcd, sql, decode_cef, and netflow modules#49643
strawgate wants to merge 7 commits intoelastic:mainfrom
strawgate:strawgate/fix-typos-pr7-obs-infraobs-integration-experience

Conversation

@strawgate
Copy link
Copy Markdown
Contributor

Summary

Fix spelling and grammar typos across several metricbeat and filebeat modules.

metricbeat (obs-infraobs-integrations):

  • metricbeat/module/http/_meta/config.yml: remove duplicated word the the
  • x-pack/metricbeat/module/iis/{application_pool,webserver,}/ _meta/docs.md: remove duplicated word the the (×3)
  • x-pack/metricbeat/module/oracle/performance/_meta/fields.yml: occuredoccurred
  • metricbeat/module/etcd/metrics/_meta/fields.yml: commitedcommitted
  • x-pack/metricbeat/module/sql/_meta/docs.md: fix grammar in module option description

filebeat (integration-experience):

  • x-pack/filebeat/processors/decode_cef/_meta/fields.yml: seperatedseparated, Thee URIThe URI
  • x-pack/filebeat/input/netflow/decoder/fields/ipfix-information-elements.csv: remove duplicated of of

Closes partial #49082
Closes partial #49335
Closes partial #49352
Closes partial #49432

Test plan

  • Verify corrected text in each _meta source file
  • Verify generated reference configs and docs match source fixes

…perience modules

metricbeat:
- http: remove duplicated 'the the' in config comment
- iis: remove duplicated 'the the' in module docs (×3 files)
- oracle: 'occured' → 'occurred' in field description
- etcd: 'commited' → 'committed' in field description
- sql: fix grammar in module docs

filebeat:
- decode_cef: 'seperated' → 'separated', 'Thee' → 'The' in field descriptions
- netflow/ipfix: remove duplicated 'of of' in CSV field data

Closes partial elastic#49082
Closes partial elastic#49335
Closes partial elastic#49352
Closes partial elastic#49432
@strawgate strawgate requested review from a team as code owners March 24, 2026 18:43
@botelastic botelastic Bot added the needs_team Indicates that the issue/PR needs a Team:* label label Mar 24, 2026
@github-actions
Copy link
Copy Markdown
Contributor

🤖 GitHub comments

Just comment with:

  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Mar 24, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: e94cb399-5797-41af-95cd-fd71897a32af

📥 Commits

Reviewing files that changed from the base of the PR and between 9675814 and 570cffe.

📒 Files selected for processing (2)
  • metricbeat/metricbeat.reference.yml
  • x-pack/metricbeat/metricbeat.reference.yml
✅ Files skipped from review due to trivial changes (2)
  • metricbeat/metricbeat.reference.yml
  • x-pack/metricbeat/metricbeat.reference.yml

📝 Walkthrough

Walkthrough

Adds a changelog fragment and corrects multiple typos and minor wording issues across Metricbeat and Filebeat documentation and metadata. Fixes include spelling corrections (e.g., “commited” → “committed”, “occured” → “occurred”, “seperated” → “separated”), removal of duplicated words (“the the”), and small phrasing edits in iis, oracle, etcd, sql, and http module docs and examples, plus decode_cef processor descriptions. Several generated asset functions now return updated base64/zlib payload strings. No public API, field names, types, or control flow were changed.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • 🛠️ Update Documentation: Commit on current branch
  • 🛠️ Update Documentation: Create PR

Comment @coderabbitai help to get the list of available commands and usage tips.

@strawgate strawgate added Team:Obs-InfraObs Label for the Observability Infrastructure Monitoring team Team:Integration-Experience backport-skip Skip notification from the automated backport with mergify labels Mar 24, 2026
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/integration-experience (Team:Integration-Experience)

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@x-pack/metricbeat/module/iis/_meta/docs.md`:
- Line 17: This change updates docs.md for the IIS module but is missing a
changelog fragment; generate a fragment using the elastic-agent-changelog-tool
and add it to the repository with the required fields: set kind (e.g.,
bugfix/documentation), a concise summary describing the docs change (mentioning
the IIS Worker Process/System counters), and component (metricbeat/iis), then
commit the generated YAML fragment so the PR includes the changelog entry.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 01dc354a-94ad-4ce5-9706-73f3ce2cdd62

📥 Commits

Reviewing files that changed from the base of the PR and between 884e58d and 7638bb7.

⛔ Files ignored due to path filters (1)
  • x-pack/filebeat/input/netflow/decoder/fields/ipfix-information-elements.csv is excluded by !**/*.csv
📒 Files selected for processing (24)
  • changelog/fragments/1774377775-fix-typos-obs-infraobs-integration-experience.yaml
  • docs/reference/metricbeat/exported-fields-etcd.md
  • docs/reference/metricbeat/exported-fields-oracle.md
  • docs/reference/metricbeat/metricbeat-metricset-iis-application_pool.md
  • docs/reference/metricbeat/metricbeat-metricset-iis-webserver.md
  • docs/reference/metricbeat/metricbeat-module-http.md
  • docs/reference/metricbeat/metricbeat-module-iis.md
  • docs/reference/metricbeat/metricbeat-module-sql.md
  • docs/reference/metricbeat/metricbeat-reference-yml.md
  • docs/reference/metricbeat/monitoring-metricbeat-collection.md
  • metricbeat/metricbeat.reference.yml
  • metricbeat/module/etcd/fields.go
  • metricbeat/module/etcd/metrics/_meta/fields.yml
  • metricbeat/module/http/_meta/config.yml
  • metricbeat/modules.d/http.yml.disabled
  • x-pack/filebeat/processors/decode_cef/_meta/fields.yml
  • x-pack/filebeat/processors/decode_cef/fields.go
  • x-pack/metricbeat/metricbeat.reference.yml
  • x-pack/metricbeat/module/iis/_meta/docs.md
  • x-pack/metricbeat/module/iis/application_pool/_meta/docs.md
  • x-pack/metricbeat/module/iis/webserver/_meta/docs.md
  • x-pack/metricbeat/module/oracle/fields.go
  • x-pack/metricbeat/module/oracle/performance/_meta/fields.yml
  • x-pack/metricbeat/module/sql/_meta/docs.md

Comment thread x-pack/metricbeat/module/iis/_meta/docs.md
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Copy link
Copy Markdown
Member

@vishaangelova vishaangelova left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@vishaangelova vishaangelova removed the backport-skip Skip notification from the automated backport with mergify label Mar 24, 2026
@vishaangelova vishaangelova added the backport-9.3 Automated backport to the 9.3 branch label Mar 24, 2026
@vishaangelova
Copy link
Copy Markdown
Member

This needs to be backported to 9.3 for the changes to be published to the docs site (we publish from the latest minor branch).

Copy link
Copy Markdown
Contributor

@pierrehilbert pierrehilbert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving but I don't think we want a changelog entry for just fixing typos.

Per reviewer feedback — typo fixes don't need changelog entries.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@strawgate strawgate enabled auto-merge (squash) April 1, 2026 13:08
@pierrehilbert
Copy link
Copy Markdown
Contributor

@ishleenk17 / @lalit-satapathy could we have someone to take a look here please?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-9.3 Automated backport to the 9.3 branch skip-changelog Team:Integration-Experience Team:Obs-InfraObs Label for the Observability Infrastructure Monitoring team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants